Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: temp ignore cached gas price #1193

Merged
merged 1 commit into from
Sep 5, 2024
Merged

fix: temp ignore cached gas price #1193

merged 1 commit into from
Sep 5, 2024

Conversation

dohaki
Copy link
Contributor

@dohaki dohaki commented Sep 5, 2024

Since the introduction of cached values, we have been seeing a lot API errors like

InputError: Relayer fill simulation failed - processing response error: SERVER_ERROR - Error: failed with 50044019 gas: max fee per gas less than block base fee: address 0x07aE8551Be970cB1cCa11Dd7a11F47Ae82e70E67, maxFeePerGas: 14025000, baseFee: 19815000
    at getRelayerFeeDetails (/vercel/path0/api/_utils.ts:632:11)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at handler (/vercel/path0/api/suggested-fees.ts:255:31)
    at async Server.<anonymous> (/opt/rust/nodejs.js:1:10566)
    at async Server.<anonymous> (/opt/rust/nodejs.js:8:4242)
END RequestId: 60797827-296d-4bde-af4a-8862b3692f0e
REPORT RequestId: 60797827-296d-4bde-af4a-8862b3692f0e	Duration: 7303.97 ms	Billed Duration: 7304 ms	Memory Size: 1024 MB	Max Memory Used: 365 MB

Most probably due to stale gas prices. Ways to circumvent these:

  • decrease TTL
  • apply multiplier to cached gas price

This PR only removes the usage of cached gas prices for now, so that we can get off the rollback deployment.

@vercel Vercel
Copy link

vercel bot commented Sep 5, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
app-frontend-v3 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 5, 2024 2:27pm
sepolia-frontend-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 5, 2024 2:27pm

@dohaki dohaki requested review from gsteenkamp89 and pxrl September 5, 2024 14:24
@dohaki dohaki merged commit abec098 into master Sep 5, 2024
9 checks passed
@dohaki dohaki deleted the ignore-cached-gas-price branch September 5, 2024 14:31
@dohaki dohaki mentioned this pull request Sep 6, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pxrl pxrl pxrl approved these changes

@gsteenkamp89 gsteenkamp89 Awaiting requested review from gsteenkamp89

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dohaki @pxrl